Skip to content

[RUN-5043] Pin container images and GitHub Actions by digest - #76

Merged
luismalamoc merged 2 commits into
mainfrom
feature/RUN-5043-pin-digests
Oct 20, 2026
Merged

luismalamoc merged 2 commits into
mainfrom
feature/RUN-5043-pin-digests

Conversation

@luismalamoc

Copy link
Copy Markdown

Summary

  • Pin third-party GitHub Actions uses: to a commit SHA, with the version in a comment.

Exceptions

  • Reusable workflows we own stay on @main (rundeck-plugins/.github snyk scan and rundeck/rundeck prepare-community-pr).

Test plan

  • CI on this pull request

@luismalamoc
luismalamoc requested review from a team and a balanced review from Copilot October 19, 2026 15:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The pins preserve the existing action versions and workflow behavior, with no blocking issues identified.

No open issues 🎉

What changed in this PR

Pins GitHub Actions to immutable commit SHAs to improve build and release supply-chain security.

Changes:

  • Pins checkout and Java setup actions in both workflows.
  • Pins artifact uploads and adds readable version comments.
File Description
.github/​workflows/​release.yml Pins checkout and Java setup actions.
.github/​workflows/​gradle.yml Pins checkout, Java setup, and artifact upload actions.

Review effort: Balanced

@Jesus-Osuna-M Jesus-Osuna-M left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving RUN-5043. Workflow-only change: checkout and Java setup go through the digest-pinned composite action. Snyk is red on existing dependency findings (not introduced by this diff).

@luismalamoc
luismalamoc merged commit 1506f31 into main Oct 20, 2026
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants